Repository navigation
fix: подтверждение исключения из команды проекта-заявки - #473
Merged
Merged
Conversation
7 tasks done
TThanos3000
deleted the
fix/dev-application-team-removal-copy-20261021
branch
October 21, 2026 10:53
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Что изменено
Окно удаления участников теперь показывает конкретные команды проектов-заявок и имена исключаемых участников. Оно объясняет, что команда исходного проекта сохраняется, а оценки уже сданной заявки не меняются. Подтверждение отправляет
confirmation_version:2.Зачем
Заявка в программе — отдельная копия проекта. Участника нужно исключать из ее команды, сохраняя исходный проект и аккаунт. Прежний текст обещал сохранить все команды и больше не соответствует выбранному поведению.
Реализация
Расширены существующие domain preview model, HTTP adapter и centered confirmation modal. Новый визуальный компонент не создается; тонкие кнопки и плавающие массовые действия сохраняются.
API
Зависит от PROCOLLAB-github/api#810. API-first deployment:
remove-previewвозвращаетconfirmation_version:2иteam_changes,removeтребуетconfirmation_version:2. Backend отклоняет старый интерфейс до его обновления, чтобы не расширять эффект удаления без нового подтверждения.UI
Показаны название заявки, участники, отзыв приглашений и сохранение оценок. Сохранено предупреждение при удалении руководителей. Проверены1440/390/360px, тонкие кнопки, centered/scrollable modal, отсутствие горизонтального overflow и сдвига строк при выборе.
Permissions
Server-side permissions и удаление выполняются API. Клиент показывает подписанный preview и отправляет подтверждение. Глобальный ProjectID и ID заявки не смешиваются.
Проверки
npm run build:prpassed.Acceptance criteria
Не проверено
Настоящие DEV users и SMTP намеренно не изменяются. Business writes в browser QA идут только в локальный API; actual DEV защищенные маршруты проверяются анонимными GET.
Риски / migration
Без миграций. Merge/deploy после успешного API PR810 deployment. Production не затрагивается.
DEV deployments completed: API PR810 →1c424ff7 (run39587159832), frontend PR473 →cc95cda8 (run39587847132). Merge trees match tested sources. Production, real users and SMTP unchanged.